Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow flexible way of banks specification #419

Merged
merged 1 commit into from
Sep 2, 2024
Merged

Conversation

Kvieta1990
Copy link
Collaborator

Description of the changes:

The previous implementation is bundled with the physical banks of NOMAD for the banks merging at the post processing stage. To expand the usage of the post processing tab to other use cases, we have to remove such a limitation.

This PR is for coping with the general way of the pixel grouping, where we can specify whatever suitable range for each banks, without worrying about the order of Q range limit for each bank that previously should have come in an increasing manner.

@Kvieta1990 Kvieta1990 merged commit fefc584 into next Sep 2, 2024
2 checks passed
@Kvieta1990 Kvieta1990 deleted the flexible_banks branch September 2, 2024 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant